Subject: draft-ietf-dlswmib-mib-09.txt comments.

From: "Mauro D. Zallocco" <[email-masked]>
Date: Fri, 21 Jun 1996 16:36:13 -0400

I have a few comments relating the notifications in 
draft-ietf-dlswmib-mib-09.txt.

The notifications in this mib list objects which are
indices to tables. There is no point in doing this because the
index information will be sent with the notification.

For dlswTrapTConnPartnerReject, dlswTConnOperPartnerVersion
dlswTConnOperPartnerVendorID and dlswTConnOperConfigIndex may be a
good choice for the OBJECTS.

dlswTrapTConnPartnerReject NOTIFICATION-TYPE
    OBJECTS { dlswTConnOperPartnerVersion, dlswTConnOperPartnerVersion,
              dlswTConnOperConfigIndex
    }
    STATUS     current
    DESCRIPTION
       "This trap is sent each time a transport connection
        is rejected by a partner DLSw during Capabilities
        Exchanges.  The emission of this trap is controlled
        by dlswTrapCntlTConnPartnerReject."
    ::= { dlswTraps 1 }

Note that dlswTConnOperTDomain and dlswTConnOperRemoteTAddr will be sent
up as part of the snmp notification.

For dlswTrapTConnProtViolation, same as for dlswTrapTConnPartnerReject.

For dlswTrapTConnUp and dlswTrapTConnDown, as well as replacing the
indices, an optimization is in order. Both can be replaced with the
following.

dlswTrapTConnUpDown NOTIFICATION-TYPE
    OBJECTS { dlswTConnOperState
    }
    STATUS     current
    DESCRIPTION
       "This trap is sent each time a transport connection
        enters `connected' or `disconnected' state.  
        dlswTConnOperState will indicate the state.
        The emission of this trap is controlled by dlswTrapCntlTConn."
    ::= { dlswTraps 3 }

General comments.

The following objects have SYNTAX specified as TimeTicks and contain
a  UNITS "hundredths of a second" clause.
The UNITS clause is redundant and can be deleted since TimeTicks is
specified as units of hundredths of a second.

dlswNodeUpTime, dlswTConnConfigLastModifyTime, dlswTConnOperEntryTime
dlswTConnOperConnectTime, dlswTConnOperDiscTime, dlswCircuitEntryTime
dlswCircuitStateTime.

/Mauro